widget: don't check before setting to avoid costly type checks
authorStefan Kost <ensonic@users.sf.net>
Thu, 11 Feb 2010 08:38:44 +0000 (10:38 +0200)
committerTristan Van Berkom <tristan.van.berkom@gmail.com>
Sun, 4 Apr 2010 00:56:13 +0000 (20:56 -0400)
There are no side effect of setting these flags and we don't fire g_object
notifies also. Thus set/unset right away without costly check of current state.

gtk/gtkwidget.c

index a9674f7d282119a23e81e517d2fa5f368e659c09..1efc66897d31b14b85fe9b95d28e78f679311a88 100644 (file)
@@ -6001,15 +6001,10 @@ gtk_widget_set_realized (GtkWidget *widget,
 {
   g_return_if_fail (GTK_IS_WIDGET (widget));
 
-  realized = realized != FALSE;
-
-  if (realized != gtk_widget_get_realized (widget))
-    {
-      if (realized)
-        GTK_WIDGET_SET_FLAGS (widget, GTK_REALIZED);
-      else
-        GTK_WIDGET_UNSET_FLAGS (widget, GTK_REALIZED);
-    }
+  if (realized)
+    GTK_WIDGET_SET_FLAGS (widget, GTK_REALIZED);
+  else
+    GTK_WIDGET_UNSET_FLAGS (widget, GTK_REALIZED);
 }
 
 /**
@@ -6048,15 +6043,10 @@ gtk_widget_set_mapped (GtkWidget *widget,
 {
   g_return_if_fail (GTK_IS_WIDGET (widget));
 
-  mapped = mapped != FALSE;
-
-  if (mapped != gtk_widget_get_mapped (widget))
-    {
-      if (mapped)
-        GTK_WIDGET_SET_FLAGS (widget, GTK_MAPPED);
-      else
-        GTK_WIDGET_UNSET_FLAGS (widget, GTK_MAPPED);
-    }
+  if (mapped)
+    GTK_WIDGET_SET_FLAGS (widget, GTK_MAPPED);
+  else
+    GTK_WIDGET_UNSET_FLAGS (widget, GTK_MAPPED);
 }
 
 /**